docs: replace Shader Playground links - #8887
Conversation
|
Azure Pipelines: There may be pipelines that require an authorized user to comment /azp run to run. |
|
@microsoft-github-policy-service agree |
There was a problem hiding this comment.
🟢 Approval recommended
The documentation-only updates were validated, with no unresolved issues.
Pull request overview
Replaces deprecated Shader Playground links in the SPIR-V Cookbook with Compiler Explorer sessions.
Changes:
- Updates introductory guidance to reference Compiler Explorer.
- Replaces all 22 obsolete URLs while preserving existing shader examples.
File summaries
| File | Description |
|---|---|
docs/SPIRV-Cookbook.rst |
Updates the service reference and interactive example links. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 0
- Review effort level: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
/azp run |
|
Azure Pipelines: Successfully started running 1 pipeline(s). |
Alex Sepkowski (alsepkow)
left a comment
Alex Sepkowski (alsepkow)
left a comment
There was a problem hiding this comment.
LGTM.
I do think the description here reads overly verbose like generated from an LLM. The 'changes' list is largely redundant and covered by the description. Similar comment on the validation list.
|
KAVYANSH KUMAR (@kavyansh18) once you update the description per Alex's comment we can merge. If you used an LLM to assist with the PR you are requirement to disclose that as part of the commit message. Thanks for the contribution! |
Description
Replaces the deprecated Shader Playground links in
docs/SPIRV-Cookbook.rstwith Compiler Explorer links.The existing HLSL examples and shader code are unchanged.
Testing